Skip to content

ADFA-5128 | Rename pair-programming-plugin to Code-Together - #81

Merged
Daniel-ADFA merged 2 commits into
mainfrom
ADFA-5128
Sep 2, 2026
Merged

ADFA-5128 | Rename pair-programming-plugin to Code-Together#81
Daniel-ADFA merged 2 commits into
mainfrom
ADFA-5128

Conversation

@Daniel-ADFA

@Daniel-ADFA Daniel-ADFA commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

The plugin carried three different names: the directory and .cgp said "pair-programming-plugin", the plugin manager said "Code Together", and the documentation heading said "Pair".

The plugin carried three different names: the directory and .cgp said
"pair-programming-plugin", the plugin manager said "Code Together", and
the documentation heading said "Pair".

Make the src directory the single source of truth and derive the rest:

  GitHub src directory   Code-Together
  Display name           Code Together
  Plugin manager name    Code Together   (plugin.name, unchanged)
  Doc HTML <h1>/<title>  Code Together
  .cgp filename          code-together.cgp
  Doc HTML filename      code-together.html

plugin.id, the Kotlin package, and the "Pair" editor-tab label are
untouched — plugin.id backs the plugin_<pluginId> tooltip category and
on-device install identity, and the tab label is a UI string, not one of
the six naming slots.

update-libs.yml now derives code-together.cgp mechanically, so no rename
map arm is needed; the previously published pair-programming.cgp website
link must be repointed.

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@Daniel-ADFA
Daniel-ADFA requested a review from a team September 1, 2026 15:20
Matches every other plugin folder in this repo, all of which are
lowercase. Deviates from the standard's MixedCase directory rule; the
effect is that the src directory and the derived slug are now the same
string rather than a lowercased copy.

Nothing else moves: the display name stays "Code Together" in
plugin.name and the doc heading, pluginName stays code-together, and
update-libs.yml still derives code-together.cgp.
@Daniel-ADFA
Daniel-ADFA merged commit b8be9ff into main Sep 2, 2026
3 checks passed
@Daniel-ADFA
Daniel-ADFA deleted the ADFA-5128 branch September 2, 2026 07:22
hal-eisen-adfa added a commit that referenced this pull request Sep 7, 2026
Two of main's ten new commits collided with this branch's move of every addon
under plugins/.

Code-Together (83 rename/rename conflicts, one cause). Main's #81 renamed
pair-programming-plugin/ to code-together/ at the repository root; this branch
renamed it to plugins/Code-Together/. Same new name, two destinations. Kept
this branch's path. Of the 83 files, 78 were byte-identical; the five that
differed are resolved as:

  README.md            main's text -- this branch still carried the
                       pre-rename copy, headed "# Pair" -- with its paths
                       moved to plugins/Code-Together, ../../gradlew,
                       ../../libs/ and ../../scripts/
  code-together.html   main's, with the same path fix, and "CodeOnTheGo"
                       written out in full: main's copy predates the rule
                       this branch added, and addons check rejected it
  settings.gradle.kts  main's rootProject.name = "code-together" over this
                       branch's stale "pair-programming-plugin", with this
                       branch's ../../libs/ classpath
  build.gradle.kts     this branch's ../../libs/; pluginName already agreed
  addon.json           this branch's; main has no gallery metadata

Favorite Snippets (one directory rename split). Main's #83 changed three files
under snippets/ while this branch moved that tree to plugins/Favorite-Snippets/
and its Kotlin package to org/appdevforall/favoritesnippets/. The two sides do
not overlap textually, so the manifest and the fragment merged on their own:
min_ide_version is main's 26.36, and the PluginWindows dialog and toast routing
sits on top of this branch's package and PLUGIN_ID. Only res/values had no
majority destination; main's new strings.xml is moved to the addon, and the
snippets/ directory is back to holding nothing but its placeholder README.

README.md: this branch's. Main's single edit renamed a table row that this
branch had already replaced.

CLAUDE.md merged with no conflict -- main added the KeystoreSecretStore
section, this branch rewrote the plugin-adding steps.

Verified: addons check passes over 22 addons, 88 tests pass, and Code-Together
builds at its new path (code-together.cgp, 6.4 MB) against main's refreshed
plugin-api.jar.

Favorite Snippets does NOT build, for a reason this merge did not cause and
cannot fix: main's #83 calls PluginWindows, which is absent from the
plugin-api.jar committed in libs/. That jar was last refreshed by 0c505c6,
before #83 landed. Demonstrated by building main's own snippets/ tree,
unmodified, against the repository's own jars: the same six "Unresolved
reference 'PluginWindows'" errors. main is broken here today; refreshing libs/
from a CodeOnTheGo build that includes plugin-api/.../base/PluginWindows.kt is
the fix, and is a change of its own.

Claude-Session: https://claude.ai/code/session_0174DaYfuVkZSJ7PbsMoicgv
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants